Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update cr.yml #36

Merged
merged 1 commit into from
Jan 9, 2024
Merged

Update cr.yml #36

merged 1 commit into from
Jan 9, 2024

Conversation

corazzon
Copy link
Contributor

@corazzon corazzon commented Jan 9, 2024

GPT 리뷰 라벨 조건을 삭제합니다. 앞으로 리뷰를 받아보고 많다고 생각되면 라벨 조건을 추가하도록 해요.

💡 Motivation and Context

여기에 왜 이 PR이 필요했는지, PR을 통해 무엇이 바뀌는지에 대해서 설명해 주세요


🔨 Modified

여기에 무엇이 크게 바뀌었는지 설명해 주세요

  • 여기에 세부 변경사항을 설명해주세요

🌟 More

  • 여기에 PR 이후 추가로 해야 할 일에 대해서 설명해 주세요


📋 커밋 전 체크리스트

  • 추가/변경에 대한 단위 테스트를 완료하였습니다.
  • 컨벤션에 맞게 작성하였습니다.

🤟🏻 PR로 완료된 이슈

closes #

GPT 리뷰 라벨 조건을 삭제합니다. 앞으로 리뷰를 받아보고 많다고 생각되면 라벨 조건을 추가하도록 해요.
Copy link

cr-gpt bot commented Jan 9, 2024

Seems you are using me but didn't get OPENAI_API_KEY seted in Variables/Secrets for this repo. you could follow readme for more information

runs-on: ubuntu-latest
steps:
- uses: anc95/ChatGPT-CodeReview@main
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
OPENAI_API_KEY: ${{ secrets.OPENAI_API_KEY }}
# optional
LANGUAGE: Korean
PROMPT: 당신은 10년차 시니어 개발자입니다. 우리가 작성한 코드에 문제가 없는지 리뷰해주세요. 대답은 한국어로 작성해주시고 단순한 부분이나 큰 이슈가 없는 부분은 리뷰해주지 않아도 되요.
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

코드 리뷰를 보면, 주석을 통해 사용 가능한 옵션 중 하나인 "if" 구문이 주석 처리되어 있습니다. "if" 구문은 제거하시는 것이 좋겠습니다. 이 부분만 제외하면 큰 문제는 없는 것으로 보입니다.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

와우 리뷰 달아주네요

Copy link
Collaborator

@chan99k chan99k left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

reopen 의 경우에도 리뷰하도록 추가한 점 확인하였습니다.
LGTM

@gunsight1
Copy link
Collaborator

확인했습니다!

Copy link
Collaborator

@linglong67 linglong67 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

저도 확인했습니다~!

@linglong67 linglong67 merged commit 3c02062 into develop Jan 9, 2024
1 check passed
@linglong67 linglong67 deleted the corazzon-patch-1 branch January 19, 2024 10:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants